Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support fog's new use_iam_profile option #203

Closed
wants to merge 1 commit into from

Conversation

fcheung
Copy link
Contributor

@fcheung fcheung commented Jun 24, 2012

EC2 recently gained the option for credentials to be fetched from the instance metadata service rather than embedding them in a credentials file. This patch allows dragonfly to do this (via fog's support for this - currently on master but unreleased)

@fcheung
Copy link
Contributor Author

fcheung commented Jan 29, 2013

This has been part of the recent releases of fog - it would be great if dragonfly was able to use it. Perhaps it would be better to make it so that dragonfly is less tightly coupled to fog? Instead of the current setup users could pass a hash of options that would be passed through verbatim to fog

@markevans
Copy link
Owner

hi - sorry for the massively late response on the original request - I agree - passing through options verbatim simplifies things a bit. I'll merge this in soon, though FYI I will also at some point move S3, Mongo and Couch datastores out of core and into their own self-contained gems
thanks again

@fcheung
Copy link
Contributor Author

fcheung commented Jan 30, 2013

That sounds like a great plan

@dpehrson
Copy link

Just wanted to pop in here and give a +1 on this. In the process of setting up the CloudFormation infrastructure for an app that will use DragonFly and IAM Instance Profiles are very attractive for credential management.

Thank to both of you for working on this, can' wait to see it merged.

@markevans
Copy link
Owner

sorry again for the delay - I've merged via a cherry-pick in master (still plan to separate out into a separate gem though!)

@markevans markevans closed this Aug 15, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants